Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add gluecose conformance tests #19

Merged
merged 3 commits into from
Mar 29, 2022

Conversation

qmuntal
Copy link
Member

@qmuntal qmuntal commented Mar 29, 2022

This PR checks in several conformance samples from https://github.com/gluecose/test-vectors and test that go-cose can sign and verify them.

Additionally, if the sample allows it, our CBOR output is compared against the sample output.

This new test suite deprecates all tests contained in sign_verify_cose_wg_examples_test.go, which were difficult to run and only did partial validations.

qmuntal added 3 commits March 29, 2022 15:55
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
Signed-off-by: qmuntal <qmuntaldiaz@microsoft.com>
@qmuntal qmuntal force-pushed the dev/qmuntal/gluecose branch from 49c9839 to 4d621e9 Compare March 29, 2022 13:55
@SteveLasker
Copy link
Contributor

Awesome seeing these. Did you mean to make two PRs? #19 and #19 ? They seem to impact the same files

@qmuntal
Copy link
Member Author

qmuntal commented Mar 29, 2022

Awesome seeing these. Did you mean to make two PRs? #19 and #19 ? They seem to impact the same files

This PR supersedes #18. I'll close it when we metge #19 .

@qmuntal
Copy link
Member Author

qmuntal commented Mar 29, 2022

@SteveLasker @thomas-fossati this is ready to merge.

@qmuntal qmuntal mentioned this pull request Mar 29, 2022
Copy link
Contributor

@thomas-fossati thomas-fossati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work, 🚢 it!

@SteveLasker SteveLasker merged commit 9a28930 into veraison:main Mar 29, 2022
@SteveLasker
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants